Skip to content

Fix the CI #62

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Oct 1, 2024
Merged

Fix the CI #62

merged 3 commits into from
Oct 1, 2024

Conversation

@bquorning
Copy link
Contributor

@bquorning bquorning commented Sep 27, 2024 •
edited
Loading

On opening #61 I realized that CI was failing for unrelated reasons, and found a few things to fix.

On the legacy builds, actions/checkout@v3 (and v4 too if you try) fail with

/__e/node20/bin/node: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.27' not found (required by /__e/node20/bin/node)
/__e/node20/bin/node: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by /__e/node20/bin/node)
/__e/node20/bin/node: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.25' not found (required by /__e/node20/bin/node)

See also actions/checkout#1590 (people reporting it broken with v3 since August 16th or so).

@bquorning bquorning marked this pull request as ready for review September 27, 2024 15:02
@bquorning bquorning mentioned this pull request Sep 27, 2024
Merged
pirj
pirj reviewed Sep 27, 2024
View reviewed changes
@pirj
Copy link
Member

pirj commented Sep 27, 2024

@JonRowe I don’t have merge rights here.

@JonRowe JonRowe merged commit cf13ee4 into rspec:main Oct 1, 2024
47 checks passed
@bquorning bquorning deleted the fix-the-ci branch October 1, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@pirj pirj pirj left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bquorning @pirj @JonRowe